Skip to content

[HOT-304948] Fix empty namespace emitted as null - #742

Merged
temaEmelyan merged 3 commits into
masterfrom
hot-304948-fix-empty-namespace-emitted-as-null
Jul 27, 2026
Merged

[HOT-304948] Fix empty namespace emitted as null#742
temaEmelyan merged 3 commits into
masterfrom
hot-304948-fix-empty-namespace-emitted-as-null

Conversation

@rboustany-crypto

Copy link
Copy Markdown
Contributor

Please make sure you consider the following:

  • Add tests that use __typename in queries
  • Does this change work with all nadel transformations (rename, type rename, hydration, etc)? Add tests for this.
  • Is it worth using hints for this change in order to be able to enable a percentage rollout?
  • Do we need to add integration tests for this change in the graphql gateway?
  • Do we need a pollinator check for this?

@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown

Test Results

  662 files  +  662    662 suites  +662   1m 6s ⏱️ + 1m 6s
2 225 tests +2 225  1 662 ✅ +1 662  563 💤 +563  0 ❌ ±0 
2 233 runs  +2 233  1 670 ✅ +1 670  563 💤 +563  0 ❌ ±0 

Results for commit a9f4e7f. ± Comparison against base commit 81a0db2.

♻️ This comment has been updated with latest results.

@rboustany-crypto
rboustany-crypto marked this pull request as ready for review July 27, 2026 00:57
@temaEmelyan
temaEmelyan merged commit 58c419e into master Jul 27, 2026
6 checks passed
@temaEmelyan
temaEmelyan deleted the hot-304948-fix-empty-namespace-emitted-as-null branch July 27, 2026 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants